Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat isolate data pull #78

Closed
wants to merge 19 commits into from
Closed

Feat isolate data pull #78

wants to merge 19 commits into from

Conversation

cpaniaguam
Copy link
Member

This PR refactors the instructions that pull data from sliderule into its own function for testing purposes.

kmilo9999 and others added 19 commits December 26, 2023 21:14
@kmilo9999
added A02c_IOWAGA_thredds_prior.py file for step 4. removed commented…
5b29d3d 
… lines and formatted
@kmilo9999
Merge branch 'main' into feat-step4-intg
733052d
@kmilo9999
added fourth command to the github workflow
29a3fcf
@kmilo9999
renamed step4 in workflow file
4678d76
@kmilo9999
fixing error in non-callable module
494245b
@kmilo9999
added siphon dependency
b43a02c
@cpaniaguam
Merge branch 'main' into feat-step4-intg
dae358f
@cpaniaguam
Merge branch 'main' into feat-step4-intg
4c486d1
@cpaniaguam
fix: step 4 cleanups
e96c943
@cpaniaguam
fix: add explicit import xarray
3ccb7c0
@cpaniaguam
fix: remove use of the imp module
309b36f 
Also, organize imports in scripts
@cpaniaguam
fix: organize imports, remove unused
f6e1580
@kmilo9999
Merge branch 'feat-step4-intg' of https://github.com/brown-ccv/icesat…
7319992 
…2-tracks into feat-step4-intg
@cpaniaguam
dep: add siphon
e703295
@cpaniaguam
Merge branch 'feat-step4-intg' of https://github.com/brown-ccv/icesat…
a6c80d3 
…2-tracks into feat-step4-intg
@kmilo9999
Merge pull request #50 from brown-ccv/feat-step4-intg
a0ee148 
Feature step4 integration
@cpaniaguam
Merge branch 'main' into feat-build-test-suite
8d895e3
@cpaniaguam
Update pytest command in run_tests.yml
3b8d17b
@cpaniaguam
feat: isolate data pull
ab96d18 
Misc: use pathlib for paths, move function to io module
@cpaniaguam cpaniaguam changed the base branch from main to feat-build-test-suite January 22, 2024 20:48
@cpaniaguam cpaniaguam closed this Jan 22, 2024
@cpaniaguam cpaniaguam mentioned this pull request Jan 31, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cpaniaguam @kmilo9999